Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cicd: increase gcp auth token lifetime #5944

Merged
merged 1 commit into from
Dec 20, 2022
Merged

cicd: increase gcp auth token lifetime #5944

merged 1 commit into from
Dec 20, 2022

Conversation

geekflyer
Copy link
Contributor

@geekflyer geekflyer commented Dec 20, 2022 •
edited
Loading

fixes 403 errors on some jobs/builds from @igor-aptos

@geekflyer
cicd: increase gcp auth token lifetime
8a4c68c 
fixes 403 errors on some jobs/builds from @igor-aptos
@geekflyer geekflyer requested a review from a team as a code owner December 20, 2022 19:54
@geekflyer geekflyer requested a review from igor-aptos December 20, 2022 19:55
@geekflyer geekflyer enabled auto-merge (squash) December 20, 2022 19:55
@geekflyer geekflyer requested a review from rustielin December 20, 2022 19:55
@geekflyer geekflyer disabled auto-merge December 20, 2022 20:42
@geekflyer geekflyer enabled auto-merge (squash) December 20, 2022 20:42
@geekflyer geekflyer disabled auto-merge December 20, 2022 20:43
@geekflyer geekflyer merged commit 109d752 into main Dec 20, 2022
@geekflyer geekflyer deleted the token-lifteime branch December 20, 2022 20:43
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@github-actions
Copy link
Contributor

✅ Forge suite land_blocking success on 8a4c68c8a2167e5ee4cd8a3a916a14a08f80e445

performance benchmark with full nodes : 6588 TPS, 5996 ms latency, 13000 ms p99 latency,(!) expired 1240 out of 2814660 txns
Test Ok

@github-actions
Copy link
Contributor

✅ Forge suite compat success on testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> 8a4c68c8a2167e5ee4cd8a3a916a14a08f80e445

Compatibility test results for testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> 8a4c68c8a2167e5ee4cd8a3a916a14a08f80e445 (PR)
1. Check liveness of validators at old version: testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b
compatibility::simple-validator-upgrade::liveness-check : 7377 TPS, 5269 ms latency, 7400 ms p99 latency,no expired txns
2. Upgrading first Validator to new version: 8a4c68c8a2167e5ee4cd8a3a916a14a08f80e445
compatibility::simple-validator-upgrade::single-validator-upgrade : 5110 TPS, 7627 ms latency, 10200 ms p99 latency,no expired txns
3. Upgrading rest of first batch to new version: 8a4c68c8a2167e5ee4cd8a3a916a14a08f80e445
compatibility::simple-validator-upgrade::half-validator-upgrade : 4509 TPS, 9045 ms latency, 11500 ms p99 latency,no expired txns
4. upgrading second batch to new version: 8a4c68c8a2167e5ee4cd8a3a916a14a08f80e445
compatibility::simple-validator-upgrade::rest-validator-upgrade : 6428 TPS, 5978 ms latency, 9600 ms p99 latency,no expired txns
5. check swarm health
Compatibility test for testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> 8a4c68c8a2167e5ee4cd8a3a916a14a08f80e445 passed
Test Ok

@Markuze Markuze mentioned this pull request Dec 26, 2022
Closed
@Markuze Markuze mentioned this pull request Jan 3, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@igor-aptos igor-aptos igor-aptos approved these changes

@rustielin rustielin rustielin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@geekflyer @rustielin @igor-aptos